Conversation
some random refactoring added IStepTitleFactory and IFluentScannerFactory
… are not used or handled by any code.
SonOfSardaar
had a problem deploying
to
Publish
GitHub Actions
Failure
added script to generate detailed coverage for local analysis. these are not meant to run on CI pipeline (that uses different summary)
removed conditionally compiled code
|
Hi @SonOfSardaar, I went through the files and had a look at all of them, most were small changes and they look good. Are you looking for any specific feedback in this PR? Otherwise, I made a couple of comments on small things I found. Thanks for the updates you made! |
Thanks @leandro-hawktech . I was waiting to see if I could get a second pair of eyes on these changes. I did not want to rush merging without good review. I also have a beta nuget published hoping i would get some comments or feedback if there were any issues. |
2 participants
some random refactoring
added IStepTitleFactory and IFluentScannerFactory to make things bit more extensible
clean up appveyor related files
added few more tests towards code coverage